thanks @jbampton , I will try to merge this asap, Have you just removed trailing spaces or also looked at tab (especially component tests have that problem)?

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2685

Hey @DaanHoogland I only looked at trailing whitespace so far.

I was going to add a GitHub Action to run flake8 to test for W291 for trailing whitespace.

By adding this check it will mean it will stop regressions.

Yes I am interested in cleaning up tab problems in future and / or other Python linting problems.

https://flake8.pycqa.org/en/latest/
https://www.flake8rules.com/rules/W291.html

@blueorangutan test

Ping @jbampton are you working on this, can you resolve conflicts? Thanks

Moving it to the next milestone

Hi @jbampton can you please fix the conflicts?

@rohityadavcloud and @nvazquez I have removed the conflicts now

@DaanHoogland I have included both spaces and tab in this update.

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: ✔️ el7 ✔️ el8 ✔️ debian ✔️ suse15. SL-JID 2913

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3665)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31260 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4687-t3665-kvm-centos7.zip
Smoke tests completed. 92 look OK, 0 have errors
Only failed tests results shown below:

Hi @DaanHoogland @weizhouapache @Pearl1594 could you please review? Smoke tests look good, no major changes besides removing spaces, I think it should be good to merge after approvals

Hey @weizhouapache GitHub Actions don't run the first time they are added to a repo with a pull request.

So this PR will need to be merged and then new PRs will run this GitHub Action.

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: ✔️ el7 ✔️ el8 ✔️ debian ✔️ suse15. SL-JID 2967

@blueorangutan test matrix

@nvazquez a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

Trillian test result (tid-3730)
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28925 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4687-t3730-vmware-65u2.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py

Trillian test result (tid-3728)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28840 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4687-t3728-xenserver-71.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py

Trillian test result (tid-3729)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28721 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4687-t3729-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py

Many thanks @jbampton PR merged, I can see the Github Action working for new PRs

Thanks everyone 💯